sync: use atomic.Bool for Once.done#73447
sync: use atomic.Bool for Once.done#73447prabhavdogra wants to merge 2 commits intogolang:masterfrom
Conversation
|
This PR (HEAD: b2702de) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/666895. Important tips:
|
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Gopher Robot: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Jorropo: Patch Set 2: Commit-Queue+1 (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Go LUCI: Patch Set 2: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2025-04-20T06:47:49Z","revision":"fd46890ba61d559f2e906ffd0bc8f8d602801407"} Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Jorropo: Patch Set 2: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Go LUCI: Patch Set 2: This CL has failed the run. Reason: Tryjob golang/try/gotip-linux-amd64_debiansid has failed with summary (view all results):
To reproduce, try Additional links for debugging: Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Go LUCI: Patch Set 2: LUCI-TryBot-Result-1 Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
This PR (HEAD: d36e0f4) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/666895. Important tips:
|
|
Message from Jorropo: Patch Set 4: Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Go LUCI: Patch Set 4: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2025-04-20T08:20:22Z","revision":"28b4b7605004139c9f516c809c5be97b63716cfe"} Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Jorropo: Patch Set 4: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Go LUCI: Patch Set 4: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Go LUCI: Patch Set 4: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Keith Randall: Patch Set 4: Auto-Submit+1 Code-Review+2 Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Go LUCI: Patch Set 4: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2025-04-21T18:01:36Z","revision":"28b4b7605004139c9f516c809c5be97b63716cfe"} Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Keith Randall: Patch Set 4: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Keith Randall: Patch Set 4: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Keith Randall: Patch Set 4: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Go LUCI: Patch Set 4: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
|
Message from Jorropo: Patch Set 4: Code-Review+2 Please don’t reply on this GitHub thread. Visit golang.org/cl/666895. |
Updated the use of atomic.Uint32 to atomic.Bool for sync package. Change-Id: Ib8da66fea86ef06e1427ac5118016b96fbcda6b1 GitHub-Last-Rev: d36e0f4 GitHub-Pull-Request: #73447 Reviewed-on: https://go-review.googlesource.com/c/go/+/666895 Reviewed-by: Junyang Shao <shaojunyang@google.com> Reviewed-by: Keith Randall <khr@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Auto-Submit: Keith Randall <khr@golang.org> Reviewed-by: Keith Randall <khr@golang.org> Reviewed-by: Jorropo <jorropo.pgm@gmail.com>
|
This PR is being closed because golang.org/cl/666895 has been merged. |
Updated the use of atomic.Uint32 to atomic.Bool for sync package.